Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adds docs about query string decoding #131

Merged
merged 1 commit into from Sep 16, 2015
Merged

Adds docs about query string decoding #131

merged 1 commit into from Sep 16, 2015

Conversation

conradkleinespel
Copy link
Contributor

@conradkleinespel conradkleinespel commented Sep 16, 2015

Hello,

I was looking for a library to decode a query string from a URL, when I hadn't realised url::form_urlencoded::parse did just that.

I hope this little doc addition makes things a bit more clear.

Best regards,
Conrad

Review on Reviewable

@SimonSapin
Copy link
Member

@SimonSapin SimonSapin commented Sep 16, 2015

Looks good.

@bors-servo r+

@bors-servo
Copy link
Contributor

@bors-servo bors-servo commented Sep 16, 2015

📌 Commit ff02193 has been approved by SimonSapin

@bors-servo
Copy link
Contributor

@bors-servo bors-servo commented Sep 16, 2015

Testing commit ff02193 with merge f8a1344...

bors-servo pushed a commit that referenced this issue Sep 16, 2015
Adds docs about query string decoding

Hello,

I was [looking](https://twitter.com/kleinespel/status/643462183783923712) for a library to decode a query string from a URL, when I hadn't realised `url::form_urlencoded::parse` did just that.

I hope this little doc addition makes things a bit more clear.

Best regards,
Conrad

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/rust-url/131)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

@bors-servo bors-servo commented Sep 16, 2015

☀️ Test successful - travis

@bors-servo bors-servo merged commit ff02193 into servo:master Sep 16, 2015
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants